Skip to content

Refactor tests, credential cache #1102

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 9 commits into from
May 25, 2023
Merged

Conversation

katcharov
Copy link
Collaborator

@katcharov katcharov commented Apr 3, 2023

JAVA-4979

Refactoring for:
JAVA-4757

@katcharov katcharov requested a review from jyemin April 3, 2023 14:17
@jyemin jyemin requested a review from DmitryLukyanov April 3, 2023 14:24
@jyemin
Copy link
Collaborator

jyemin commented Apr 3, 2023

Adding @DmitryLukyanov to take first crack at this.

Edit: never mind. This is just refactoring, it seems.

@jyemin jyemin removed the request for review from DmitryLukyanov April 3, 2023 14:29
@katcharov katcharov requested a review from jyemin April 5, 2023 16:56
@jyemin
Copy link
Collaborator

jyemin commented Apr 20, 2023

I can't remember whether we intend to merge this or just close it.

@katcharov katcharov force-pushed the JAVA-4757-cleanup branch from 9ee986c to ad9b6c2 Compare May 18, 2023 16:15
@jyemin jyemin requested a review from stIncMale May 18, 2023 16:30
@katcharov katcharov force-pushed the JAVA-4757-cleanup branch 2 times, most recently from 5536610 to d562cb3 Compare May 18, 2023 16:40
@katcharov katcharov force-pushed the JAVA-4757-cleanup branch from d562cb3 to d46f122 Compare May 18, 2023 16:42
@stIncMale stIncMale self-requested a review May 19, 2023 21:10
Copy link
Member

@stIncMale stIncMale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved, but before merging, we need to make sure that the static check and tests pass.

Copy link
Member

@stIncMale stIncMale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved, but before merging, we need to make sure that the static check and tests pass.

Update: what an interesting duplication ☝️

@katcharov katcharov merged commit c9cfd8e into mongodb:master May 25, 2023
@katcharov katcharov deleted the JAVA-4757-cleanup branch May 25, 2023 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants